Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AE-157 Documentation for Strategy Resolver #3047

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

gerner-spryker
Copy link
Collaborator

@gerner-spryker gerner-spryker commented Feb 6, 2025

PR Description

Spryker introduced Strategy Resolver concept for enabling context specific resolution for widely overarching workflows.

Tickets

Checklist

  • I agree with the Code Contribution License Agreement in CONTRIBUTING.md

@gerner-spryker gerner-spryker added new PR contains a new document tech review needed PR needs tech review do not merge PR is not ready for merge labels Feb 6, 2025
@gerner-spryker
Copy link
Collaborator Author

gerner-spryker commented Feb 6, 2025

The PR needs to be ready for merge, but only published when the related feature is released (will be contacted by a Product member).

@gerner-spryker gerner-spryker added TW review needed PR needs Technical writer's review and removed tech review needed PR needs tech review labels Feb 6, 2025

---

Spryker introduces context-based dependency resolution, such as plugin-stack, using a strategy resolver to handle complex workflows that stretch across multiple modules. This enhancement allows for defining multiple dependencies and selecting the appropriate one dynamically at runtime by the business logic.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's an appropriate dependency?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So, 1 plugin stack that uses strategy resolving, has multiple context. Each context is basically a separate individual plugin stack (with many plugins). When you seelct a context, you actually say which plugin stack to execute.

feversocial
feversocial previously approved these changes Feb 24, 2025



# Context-Based Dependency Resolution in Spryker
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @gerner-spryker and @andriitserkovnyi
The whole below part looks like a duplicate of the above. Not a full copy-paste though but I don't get how is it different from the above

If you decide to merge it with the above - there's additional Q&A point that @profuel has added, don't forget it

@matweew matweew removed the do not merge PR is not ready for merge label Feb 27, 2025
@matweew
Copy link
Collaborator

matweew commented Feb 27, 2025

the feature is already released https://github.com/spryker/kernel/releases/tag/3.77.0, so documentation should be also released

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new PR contains a new document TW review needed PR needs Technical writer's review
Development

Successfully merging this pull request may close these issues.

4 participants